Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

test: setup goleak testing #1164

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from
Draft

test: setup goleak testing #1164

wants to merge 1 commit into from
+14 −0

Conversation

moul
Copy link
Member

@moul moul commented Sep 22, 2023

Signed-off-by: moul 94029+moul@users.noreply.github.com

Contributors' checklist...
  • Added new tests, or not needed, or not feasible
  • Provided an example (e.g. screenshot) to aid review or the PR is self-explanatory
  • Updated the official documentation or not needed
  • No breaking changes were made, or a BREAKING CHANGE: xxx message was included in the description
  • Added references to related issues and PRs
  • Provided any useful hints for running manual tests
  • Added new benchmarks to generated graphs, if any. More info here.

@moul
test: setup goleak testing
c57e2ba 
Signed-off-by: moul <94029+moul@users.noreply.github.com>
@moul moul self-assigned this Sep 22, 2023
@github-actions github-actions bot added the 📦 ⛰️ gno.land Issues or PRs gno.land package related label Sep 22, 2023
@codecov
Copy link

codecov bot commented Sep 22, 2023 •
edited
Loading

Codecov Report

Patch coverage has no change and project coverage change: -0.06% ⚠️

Comparison is base (61f3ada) 47.08% compared to head (c57e2ba) 47.02%.

Additional details and impacted files 
@@            Coverage Diff             @@
##           master    #1164      +/-   ##
==========================================
- Coverage   47.08%   47.02%   -0.06%     
==========================================
  Files         365      365              
  Lines       61156    61156              
==========================================
- Hits        28793    28760      -33     
- Misses      30011    30044      +33     
  Partials     2352     2352
Flag Coverage Δ
gno.land-_test.gnokey 0.00% <ø> (ø)
gno.land-_test.pkgs 27.88% <ø> (ø)
gnovm-_test.cmd 45.89% <ø> (ø)
gnovm-_test.gnolang.native 63.09% <ø> (ø)
gnovm-_test.gnolang.other 16.63% <ø> (ø)
gnovm-_test.gnolang.pkg0 17.98% <ø> (ø)
gnovm-_test.gnolang.pkg1 8.21% <ø> (ø)
gnovm-_test.gnolang.pkg2 9.87% <ø> (ø)
gnovm-_test.gnolang.realm 41.68% <ø> (ø)
gnovm-_test.gnolang.stdlibs 53.53% <ø> (ø)
gnovm-_test.pkg 25.96% <ø> (ø)
tm2-_test.pkg.amino 58.32% <ø> (ø)
tm2-_test.pkg.bft 63.64% <ø> (-0.29%) ⬇️
tm2-_test.pkg.others 59.24% <ø> (+<0.01%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

see 6 files with indirect coverage changes

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@moul moul added this to the 🌟 main.gno.land (wanted) milestone Sep 22, 2023
@moul
Copy link
Member Author

moul commented Oct 9, 2023

Related with #1215.

@moul moul mentioned this pull request Oct 9, 2023
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@zivkovicmilos zivkovicmilos Awaiting requested review from zivkovicmilos zivkovicmilos will be requested when the pull request is marked ready for review zivkovicmilos is a code owner

At least 1 approving review is required to merge this pull request.

Assignees

@moul moul

Labels
📦 ⛰️ gno.land Issues or PRs gno.land package related
Projects
😎 Manfred's Board
Status: 🙏🏻 To Delegate
🚀 The Launch [DEPRECATED]
Status: 🌟 Wanted for Launch
Milestone
🌟 main.gno.land (wanted)
Development

Successfully merging this pull request may close these issues.
1 participant
@moul